Content.Server/_White/Headcrab/HeadcrabSystem.cs (2)

1-30: Рекомендация: убрать дублирование подключений
Тип Content.Server.NPC.Components импортируется как в строке 3, так и в строке 7. Это потенциально избыточно и может быть упрощено, чтобы избежать путаницы.

---

127-137: Реализация укуса при экипировке в руку (OnGotEquippedHand)
Код корректно проверяет, жив ли хедкраб, не является ли пользователь зомби, а затем наносит урон и отображает всплывающие сообщения. Можно рассмотреть единообразие поведения с другими методами экипировки, но в целом решение работает.

Resources/Locale/en-US/_white/popups/headcrab.ftl (2)

4-5: Обратите внимание на согласованность использования заглавных букв при работе с плейсхолдерами.

Вместо { CAPITALIZE($entity) } можно рассмотреть более гибкий способ форматирования, если потребуется отображать другие элементы строки в верхнем регистре. В остальном логика корректна.

---

11-11: Уточните формулировку «You cant take headcrab off your head.»

Добавьте апостроф в слове "can't" для корректного написания.

- headcrab-try-unequip = You cant take headcrab off your head.
+ headcrab-try-unequip = You can't take headcrab off your head.

Content.Server/_White/Headcrab/HeadcrabComponent.cs (3)

21-23: Проверьте возможную опечатку в названии переменной «ChansePounce».

Может имелось в виду «ChancePounce»? Уточните для единообразия и читаемости кода.

- public int ChansePounce = 33;
+ public int ChancePounce = 33;

---

28-28: Именование флага IsDeath может сбивать с толку.

Обычно в подобных случаях используется IsDead. Переименуйте для большей ясности.

- public bool IsDeath = false;
+ public bool IsDead = false;

---

30-30: Переменная EquipedOn может содержать ошибку правописания.

Рекомендуется заменить на EquippedOn для согласованности.

- public EntityUid EquipedOn;
+ public EntityUid EquippedOn;

Resources/Textures/_White/Mobs/Animals/headcrab.rsi/meta.json (1)

9-32: Проверка состояний спрайта

Рекомендуется добавить дополнительные состояния анимации:

• Состояние прыжка (jumping)
• Состояние приземления (landing)
• Состояние атаки (attacking)

Это улучшит визуальную обратную связь для игроков.

Resources/ServerInfo/_White/Guidebook/Antagonist/Headcrab/Headcrabs/HeadcrabFast.xml (2)

2-3: Форматирование заголовка

Рекомендуется использовать теги форматирования для заголовка:

-  # Быстрый хедкраб
+  <Box>
+    <Heading>Быстрый хедкраб</Heading>
+  </Box>

---

15-17: Форматирование характеристик

Рекомендуется использовать список с маркерами для лучшей читаемости:

-  - Наносит [color=#ff1f1f]2 режущего[/color] своими челюстями, в том числе при закреплении.
-  - Имеет 50 ХП.
-  - Передвигается на 20% быстрее.
+  <Box>
+    <GuideReagentEmbed Reagent="Damage"/>
+    <List>
+      <ListItem>Наносит [color=#ff1f1f]2 режущего[/color] своими челюстями, в том числе при закреплении.</ListItem>
+      <ListItem>Имеет 50 ХП.</ListItem>
+      <ListItem>Передвигается на 20% быстрее.</ListItem>
+    </List>
+  </Box>

Resources/Prototypes/_White/Entities/Structures/Other/headcrab_nest.yml (1)

11-17: Рекомендуется усилить уязвимость к огню

Для биологической сущности урон от огня (10) кажется заниженным. Рекомендуется увеличить урон от огня для баланса.

  - type: Flammable
    fireSpread: true
    damage:
     types:
-       Heat: 10
+       Heat: 25

Resources/Prototypes/_White/Entities/Markers/Spawners/headcrab.yml (1)

1-55: Рекомендуется добавить условия спавна

Текущая реализация спавнеров не учитывает условия окружающей среды. Рекомендуется добавить проверки для более сбалансированного геймплея.

Предлагаемые дополнения для каждого спавнера:

  - type: ConditionalSpawner
    prototypes:
      - HeadcrabNest
+    conditions:
+      - !type:AmbientTemperature
+        min: 280
+        max: 320
+      - !type:NotInSpace
+      - !type:Distance
+        target: Players
+        minimum: 10

Resources/ServerInfo/_White/Guidebook/Antagonist/Headcrab/Headcrabs/HeadcrabPoison.xml (2)

13-17: Рекомендуется дополнить информацию о механиках

В разделе особенностей следует добавить информацию о контрмерах и тактике борьбы с ядовитыми хедкрабами.

  ## Особенности
  
  - Наносит [color=#46A446]2 яда[/color] своими челюстями, в том числе при закреплении.
  - Имеет 50 ХП.
  - Передвигается на 40% медленее.
+ - [color=#FF0000]Внимание:[/color] Уязвим к:
+   - Огнестрельному оружию
+   - Энергетическому оружию
+   - Взрывчатке
+ - [color=#FFD700]Совет:[/color] Держитесь на расстоянии и атакуйте группой

---

2-2: Рекомендуется использовать стандартный формат заголовка

-  # Ядовитый хедкраб
+  <Title>Ядовитый хедкраб</Title>

Resources/ServerInfo/_White/Guidebook/Antagonist/Headcrab/Headcrabs/Headcrab.xml (1)

4-6: Рекомендуется улучшить форматирование документации

Для улучшения читаемости рекомендуется:

• Добавить атрибут Scale для GuideEntityEmbed
• Добавить описательный Caption

  <Box>
-   <GuideEntityEmbed Entity="MobHeadcrab" Caption=""/>
+   <GuideEntityEmbed Entity="MobHeadcrab" Scale="1.5" Caption="Обычный хедкраб"/>
  </Box>

Resources/Prototypes/_White/Entities/Mobs/NPCs/headcrab.yml (4)

33-33: Рекомендации по балансировке параметров

Текущие параметры могут сделать хедкрабов слишком сильными:

• Время паралича (3 секунды) кажется слишком коротким
• Базовая скорость передвижения (2.9/3.9) выглядит высокой

Рекомендуется протестировать следующие значения:

-    paralyzeTime: 3
+    paralyzeTime: 5
-    baseWalkSpeed: 2.9
-    baseSprintSpeed: 3.9
+    baseWalkSpeed: 2.5
+    baseSprintSpeed: 3.5

Also applies to: 86-87

---

115-117: Раскомментируйте секцию GuideHelp

Документация важна для понимания механик игры. Рекомендуется раскомментировать секцию GuideHelp для базового хедкраба.

-#  - type: GuideHelp
-#    guides:
-#    - Headcrab
+  - type: GuideHelp
+    guides:
+    - Headcrab

---

144-151: Усильте урон ядовитого хедкраба

Текущий урон от яда (2 единицы) кажется слишком низким для специализированного ядовитого варианта хедкраба.

    damage:
      types:
-        Poison: 2
+        Poison: 4
  - type: MeleeWeapon
    damage:
      types:
-        Poison: 2
+        Poison: 4

---

1-157: Исправьте проблемы с отступами

Статический анализ выявил несколько проблем с отступами в файле. Рекомендуется использовать консистентные отступы по 4 пробела для всех уровней вложенности.

🧰 Tools

🪛 yamllint (1.35.1)

[warning] 16-16: wrong indentation: expected 4 but found 2

(indentation)

---

[error] 20-20: trailing spaces

(trailing-spaces)

---

[warning] 90-90: wrong indentation: expected 6 but found 4

(indentation)

---

[warning] 95-95: wrong indentation: expected 6 but found 4

(indentation)

---

[warning] 97-97: wrong indentation: expected 6 but found 4

(indentation)

---

[warning] 111-111: wrong indentation: expected 4 but found 2

(indentation)

---

[warning] 124-124: wrong indentation: expected 4 but found 2

(indentation)

---

[warning] 140-140: wrong indentation: expected 4 but found 2

(indentation)

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

#!/bin/bash
# Description: Check if all referenced XML files exist

for file in \
  "ServerInfo/_White/Guidebook/Antagonist/Headcrab/Headcrabs/Headcrab.xml" \
  "ServerInfo/_White/Guidebook/Antagonist/Headcrab/Headcrabs/HeadcrabFast.xml" \
  "ServerInfo/_White/Guidebook/Antagonist/Headcrab/Headcrabs/HeadcrabPoison.xml"
do
  if [ ! -f "Resources/$file" ]; then
    echo "Missing file: $file"
  fi
done

#!/bin/bash
# Let's try using fd to find XML files related to headcrabs
fd -e xml -p "Headcrab.*\.xml$" Resources/ServerInfo

#!/bin/bash
# Also check the general structure of ServerInfo directory
fd . Resources/ServerInfo

#!/bin/bash
# Check if the directory exists
ls -la Resources/ServerInfo/_White/Guidebook/Antagonist/Headcrab/Headcrabs/ 2>/dev/null || echo "Directory not found"

#!/bin/bash
# Проверка существования прототипа SpawnHeadcrabNest
rg -l "id:\s*SpawnHeadcrabNest" "Resources/Prototypes"

#!/bin/bash
# Проверяем содержимое файла с прототипом SpawnHeadcrabNest
cat "Resources/Prototypes/_White/Entities/Markers/Spawners/headcrab.yml"

# Проверяем веса других событий для сравнения
rg "weight:\s+\d+" "Resources/Prototypes/_White/GameRules/events.yml" -A 1

#!/bin/bash
# Проверка локализации и ресурсов
echo "Checking localizations..."
rg "jump-action-(name|description)" "Resources/Locale"

echo "Checking icon existence..."
fd "headcrab.rsi" "Resources"

Content.Server/_White/Headcrab/HeadcrabComponent.cs (1)

24-26: Требуется документация для параметра Damage

Добавьте XML-документацию, описывающую структуру и назначение спецификации урона.

+/// <summary>
+/// Определяет типы и количество урона, наносимого хедкрабом
+/// </summary>
 [DataField("damage", required: true)]
 [ViewVariables(VVAccess.ReadWrite)]
 public DamageSpecifier Damage = default!;

Content.Server/_White/Headcrab/HeadcrabSystem.cs (2)

3-3: Удалите дублирующийся импорт

Импорт Content.Server.NPC.Components дублируется.

 using Content.Server.NPC.Components;
 using Content.Server.Nutrition.Components;
 using Content.Server.Nutrition.EntitySystems;
 using Content.Server.Popups;
-using Content.Server.NPC.Components;
 using Content.Server.NPC.Systems;

Also applies to: 7-7

---

223-254: Оптимизируйте метод Update

Метод Update содержит вложенную логику, которую можно упростить:

1. Используйте раннее возвращение для уменьшения уровня вложенности
2. Выделите логику обработки урона в отдельный метод

 public override void Update(float frameTime)
 {
     base.Update(frameTime);
+    foreach (var comp in EntityQuery<HeadcrabComponent>())
+    {
+        UpdateHeadcrabDamage(comp, frameTime);
+    }
 }

+private void UpdateHeadcrabDamage(HeadcrabComponent comp, float frameTime)
+{
+    comp.Accumulator += frameTime;
+    if (comp.Accumulator <= comp.DamageFrequency)
+        return;
+
+    comp.Accumulator = 0;
+    if (comp.EquippedOn is not { Valid: true } targetId)
+        return;
+
+    if (HasComp<ZombieComponent>(comp.EquippedOn))
+        return;
+
+    if (!TryProcessTargetState(comp, targetId))
+        return;
+
+    ApplyDamageAndEffects(comp, targetId);
+}

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

#!/bin/bash
# Поиск других сущностей с похожим уроном для сравнения
rg -A 2 "Piercing:" "Resources/Prototypes/_White/Entities/Mobs/"

#!/bin/bash
# Поиск других типов урона у мобов для сравнения
rg -A 2 "damage:" "Resources/Prototypes/_White/Entities/Mobs/"

#!/bin/bash
# Поиск всех файлов мобов для полного анализа
fd . "Resources/Prototypes/_White/Entities/Mobs/" -e yml -x cat {}

Content.Server/_White/Headcrab/HeadcrabSystem.cs (1)

156-196: ⚠️ Potential issue

Оптимизируйте обработку попаданий

Метод OnMeleeHit использует return внутри цикла, что может привести к пропуску обработки других целей. Рекомендуется использовать continue для корректной обработки всех целей.

 foreach (var entity in args.HitEntities)
 {
     if (!HasComp<HumanoidAppearanceComponent>(entity))
-        return;
+        continue;

     if (TryComp(entity, out MobStateComponent? mobState))
     {
         if (mobState.CurrentState is not MobState.Alive)
         {
-            return;
+            continue;
         }
     }

Content.Server/_White/Headcrab/HeadcrabSystem.cs (2)

3-3: Удалите дублирующийся импорт

Импорт Content.Server.NPC.Components объявлен дважды. Удалите дублирующуюся строку.

using Content.Server.NPC.Components;
-using Content.Server.NPC.Components;

Also applies to: 7-7

---

90-99: Оптимизируйте отображение сообщений

Три похожих вызова PopupEntity можно объединить в один метод для уменьшения дублирования кода.

-_popup.PopupEntity(Loc.GetString("headcrab-hit-entity-head"),
-    args.Target, args.Target, PopupType.LargeCaution);
-
-_popup.PopupEntity(Loc.GetString("headcrab-hit-entity-head",
-        ("entity", args.Target)),
-    uid, uid, PopupType.LargeCaution);
-
-_popup.PopupEntity(Loc.GetString("headcrab-eat-other-entity-face",
-    ("entity", args.Target)), args.Target, Filter.PvsExcept(uid), true, PopupType.Large);
+private void ShowHeadcrabAttackPopups(EntityUid headcrab, EntityUid target)
+{
+    _popup.PopupEntity(Loc.GetString("headcrab-hit-entity-head"), 
+        target, target, PopupType.LargeCaution);
+    _popup.PopupEntity(Loc.GetString("headcrab-hit-entity-head", ("entity", target)),
+        headcrab, headcrab, PopupType.LargeCaution);
+    _popup.PopupEntity(Loc.GetString("headcrab-eat-other-entity-face", ("entity", target)),
+        target, Filter.PvsExcept(headcrab), true, PopupType.Large);
+}

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

 if (HasComp<ZombieComponent>(comp.EquippedOn))
-    return;
+    continue;

Content.Server/_White/Headcrab/HeadcrabSystem.cs (4)

77-88: Заблокированная попытка снять хедкраба.
Текущая логика правильно отменяет снятие при исполнении ряда условий, однако стоит явно сообщать игрокам причины отмены (например, если зомби). Сейчас всплывающее сообщение одно для всех случаев.

---

90-104: Проверка смерти и состояния приведения дублируется.
В методе OnGotEquippedHand есть две одинаковые проверки _mobState.IsDead(uid) || HasComp<ZombieComponent>(args.User). Можно объединить их в один блок, чтобы упростить код и повысить читаемость.

---

140-142: Случайный шанс на прыжок.
Использование _random.Next(1, 101) <= component.ChancePounce корректно, но если вы планируете расширять механику, лучше вынести генерацию шанса в отдельный метод и покрыть тестами.

---

144-146: Избыточная двойная проверка.
Сначала мы делаем TryEquip, а позже опять проверяем условия для экипировки. Есть риск дублирования логики. Объедините эти проверки, чтобы избежать путаницы.

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

Content.Server/_White/Headcrab/HeadcrabSystem.cs (3)

115-163: ⚠️ Potential issue

Оптимизируйте метод OnMeleeHit

В методе есть несколько проблем:

1. Использование return вместо continue в цикле прерывает обработку остальных сущностей
2. Дублирование кода проверок с другими методами

Рекомендуется:

1. Заменить return на continue
2. Вынести повторяющиеся проверки в отдельный метод

- if (!HasComp<HumanoidAppearanceComponent>(entity))
-     return;
+ if (!HasComp<HumanoidAppearanceComponent>(entity))
+     continue;

- private void OnMeleeHit(EntityUid uid, HeadcrabComponent component, MeleeHitEvent args)
+ private bool CanAttachToTarget(EntityUid headcrab, EntityUid target, HeadcrabComponent component)
+ {
+     return HasComp<HumanoidAppearanceComponent>(target) 
+         && _mobState.IsAlive(target)
+         && !HasComp<ZombieComponent>(target)
+         && !_mobState.IsDead(headcrab);
+ }

---

199-204: ⚠️ Potential issue

Исправьте преждевременный выход из цикла

Использование return вместо continue приводит к преждевременному завершению обработки всех хедкрабов. Это может привести к пропуску обновления состояния других хедкрабов.

 if (!_mobState.IsAlive(targetId))
 {
     _inventory.TryUnequip(targetId, "mask", true, true);
     comp.EquippedOn = EntityUid.Invalid;
-    return;
+    continue;
 }

---

206-206: 🛠️ Refactor suggestion

Добавьте обработку результата TryChangeDamage

Необходимо проверять результат вызова TryChangeDamage для корректной обработки случаев, когда урон не может быть применен.

- _damageableSystem.TryChangeDamage(targetId, comp.Damage);
+ if (!_damageableSystem.TryChangeDamage(targetId, comp.Damage, out var appliedDamage))
+ {
+     _popup.PopupEntity(Loc.GetString("headcrab-damage-resist"), targetId);
+ }

Content.Server/_White/Headcrab/HeadcrabSystem.cs (1)

38-50: Добавьте проверку зависимостей

Рекомендуется добавить проверку инжектированных зависимостей в конструкторе системы для предотвращения NullReferenceException во время выполнения.

+ public HeadcrabSystem()
+ {
+     ArgumentNullException.ThrowIfNull(_transform);
+     ArgumentNullException.ThrowIfNull(_stunSystem);
+     // ... проверки для остальных зависимостей
+ }

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

+ private const float THROW_SPEED = 7f;
+ private const float THROW_ANGULAR_VELOCITY = 10f;

- _throwing.TryThrow(uid, direction, 7F, uid, 10F);
+ _throwing.TryThrow(uid, direction, THROW_SPEED, uid, THROW_ANGULAR_VELOCITY);